-
-
Notifications
You must be signed in to change notification settings - Fork 28.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove states UI options in group integration #32021
Conversation
Hey there @home-assistant/core, mind taking a look at this pull request as its been labeled with a integration ( |
@@ -257,7 +244,7 @@ def groups_with_entity(hass: HomeAssistantType, entity_id: str) -> List[str]: | |||
|
|||
extra_arg = { | |||
attr: service.data[attr] | |||
for attr in (ATTR_VISIBLE, ATTR_ICON, ATTR_VIEW, ATTR_CONTROL) | |||
for attr in (ATTR_ICON) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now you're iterating over each character in the string, one comma is all the difference…
for attr in (ATTR_ICON) | |
for attr in (ATTR_ICON,) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we on top of this, to fix it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, seeing the notification just now. Adjusting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why I merged this without being addressed 🤦♂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before coffee ☕️ 😉
All good now 👍
Breaking change
Group configurations options, services and service options related to the old states UI were deprecated in 0.105 and now removed.
This includes:
view
andcontrol
configuration options for a group.group.set_visibility
service callcontrol
,visible
,view
options on thegroup.set
service call.Please ensure your configuration and automations don't use these anymore.
Proposed change
Remove states UI options in group integration, deprecated in 0.105: #30831
Documentation was already removed on initial deprecation.
Type of change
Example entry for
configuration.yaml
:# Example configuration.yaml
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale: